home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c++-part2 / 11485 < prev    next >
Encoding:
Text File  |  1996-08-05  |  2.4 KB  |  81 lines
  1. Path: EU.net!sun4nl!ittpub!ittpub!nntp
  2. Newsgroups: comp.lang.c++
  3. Subject: Re: Lifetime of temporary parameter objects
  4. Message-ID: <1996Mar14.131613.1805@ittpub>
  5. From: wil@ittpub.nl (Wil Evers)
  6. Date: 14 Mar 96 13:16:13 WET
  7. References: <4i6607$l4q@damon.irf.uni-dortmund.de>
  8. Distribution: world
  9. Nntp-Posting-Host: lintilla
  10.  
  11. In article <4i6607$l4q@damon.irf.uni-dortmund.de>  
  12. rothert@damon.irf.uni-dortmund.de (Bernd Rothert) writes:
  13.  
  14. > I need some advice concerning the lifetime of temporary parameter
  15. > objects infunction calls:
  16. > 
  17. > It is obviously correct to pass a reference to a temporary object to a
  18. > function - the compiler ensures that the temporary stays alive as long
  19. > as the reference exists (actual parameter of "const Stars&").
  20. > 
  21. > What if passing a pointer to the CONTENTS of an explicitly constructed
  22. > temporary:
  23. > 
  24. > ---
  25. > #include <string.h>
  26. > #include <iostream.h>
  27. > class Stars {
  28. >   public:
  29. >     Stars(int n) : _s(new char[n+1]) {
  30. >         memset(_s, '*', n);
  31. >         _s[n] = '\0';
  32. >     }
  33. >     ~Stars() {
  34. >         delete _s;
  35.         ^--- that should be delete[] _s;
  36. >         _s = 0;
  37. >     }
  38. >     const char* get() const { return _s; }
  39. >   private:
  40. >     char* _s;
  41. > };
  42. > void banner(const Stars& st, const char* s) {
  43. >                //=============
  44. >     cout << st.get() << endl << s << endl;
  45. > }
  46. > int main() {
  47. >     banner(40, Stars(40).get());
  48. >          //===============
  49. >     return 0;
  50. > }
  51. > ---
  52. > 
  53. > The program passes a pointer to the "_s" member of a "Stars" object
  54. > to the "banner" function. This works for every compiler I have seen but
  55. > I don't know if I can rely on this behaviour and prefer defining an
  56. > auxiliary variable which is guaranteed to live until the function call
  57. > returns.
  58. > 
  59. > Can anyone give me a hint?
  60.  
  61. The older rules as defined in the ARM (1990) allowed the temporary Stars  
  62. object created in `Stars(40).get()' to be destroyed *before* control was  
  63. passed to banner(). Older versions of GNU C++ are known to do this. The  
  64. draft ANSI/ISO standard effectively says the temporary will be destructed  
  65. at the end of the statement, that is, after the call to banner(). But note  
  66. that even the new rules may surprise us:
  67.  
  68. void f()
  69. {
  70.     char *p = Stars(40).get();
  71.     cout << p << endl;    // has undefined result
  72. }
  73.  
  74. So if you have a class that passes a pointer to some data in its internal  
  75. representation, I would always recommend using an explictly named  
  76. variable.
  77.  
  78. Hope this helps,
  79.  
  80. - Wil
  81.